-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
added adapters for gjirafa and malltv #5587
Conversation
Docs PR prebid/prebid.github.io#2200 |
@codewiz14 I'm looking at this and things generally look good, but your endpoint is refusing test connections. We need 100% fill sample bid params so we can see bids to verify adapters and so that publishers can use it to verify their own builds and integrations work. This is what I'm seeing:
I've tried both the params here and in your doc PR and neither is returning bids. |
Hi @GLStephen, it was configured to work only for |
@codewiz14 the gjirafa bidder looks good, but the malltv bidder is receiving an error here:
The error is:
I suspect a required parameter may have changed in V4. |
@GLStephen the issue was on interpret response when there is no bid, so we just updated |
@codewiz14 I missed the window for the release today, but this will be in the next release |
* added adapters for gjirafa and malltv * interpretResponse fix for empty result * updated testing propertyId and placementId
This reverts commit 134ecdf.
Type of change
Description of change